Skip to content

refactor: suppress unnecessary null assertion in generated Kotlin files and update pbandk version [WPB-8645]#4158

Merged
MohamadJaara merged 1 commit into
developfrom
mo/refactor/supress-generated-proto-classes-null-assertion
May 22, 2026
Merged

refactor: suppress unnecessary null assertion in generated Kotlin files and update pbandk version [WPB-8645]#4158
MohamadJaara merged 1 commit into
developfrom
mo/refactor/supress-generated-proto-classes-null-assertion

Conversation

@MohamadJaara
Copy link
Copy Markdown
Member

@MohamadJaara MohamadJaara commented May 21, 2026

https://wearezeta.atlassian.net/browse/WPB-8645

the generated proto mapping classes have unnecessary null assertion with a wall of warnings

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

Test Results

0 tests   - 4 941   0 ✅  - 4 826   0s ⏱️ - 2m 40s
0 suites  -   810   0 💤  -   115 
0 files    -   810   0 ❌ ±    0 

Results for commit 29da51d. ± Comparison against base commit e205957.

♻️ This comment has been updated with latest results.

@MohamadJaara MohamadJaara changed the title refactor: suppress unnecessary null assertion in generated Kotlin files and update pbandk version refactor: suppress unnecessary null assertion in generated Kotlin files and update pbandk version [WPB-8645] May 21, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.60%. Comparing base (e205957) to head (29da51d).

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #4158   +/-   ##
==========================================
  Coverage      61.60%   61.60%           
  Complexity      4028     4028           
==========================================
  Files           2073     2073           
  Lines          67742    67742           
  Branches        6685     6685           
==========================================
  Hits           41735    41735           
  Misses         23346    23346           
  Partials        2661     2661           

see 7 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e205957...29da51d. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown
Contributor

🐰 Bencher Report

Branchmo/refactor/supress-generated-proto-classes-null-assertion
Testbedubuntu-latest

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds flag.

Click to view all benchmark results
BenchmarkLatencymicroseconds (µs)
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInFiles📈 view plot
⚠️ NO THRESHOLD
891.63 µs
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInMemory📈 view plot
⚠️ NO THRESHOLD
325,417.18 µs
com.wire.kalium.benchmarks.persistence.MessageReadBenchmark.inboxPagingDeepPageBenchmark📈 view plot
⚠️ NO THRESHOLD
132,837.70 µs
com.wire.kalium.benchmarks.persistence.MessageReadBenchmark.inboxPagingFirstPageBenchmark📈 view plot
⚠️ NO THRESHOLD
127,543.20 µs
com.wire.kalium.benchmarks.persistence.MessageReadBenchmark.localMarkAsReadBenchmark📈 view plot
⚠️ NO THRESHOLD
3,517.94 µs
com.wire.kalium.benchmarks.persistence.MessageReadBenchmark.messagePagingDeepPageBenchmark📈 view plot
⚠️ NO THRESHOLD
26,693.25 µs
com.wire.kalium.benchmarks.persistence.MessageReadBenchmark.messagePagingFirstPageBenchmark📈 view plot
⚠️ NO THRESHOLD
11,532.69 µs
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.messageInsertionBenchmark📈 view plot
⚠️ NO THRESHOLD
1,333,464.55 µs
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.queryMessagesBenchmark📈 view plot
⚠️ NO THRESHOLD
21,353.18 µs
🐰 View full continuous benchmarking report in Bencher

@MohamadJaara MohamadJaara enabled auto-merge May 21, 2026 13:28
@MohamadJaara MohamadJaara added this pull request to the merge queue May 22, 2026
Merged via the queue into develop with commit 76c8631 May 22, 2026
29 of 30 checks passed
@MohamadJaara MohamadJaara deleted the mo/refactor/supress-generated-proto-classes-null-assertion branch May 22, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants